I like the new KeyValuesCmd name, but I think the old API was cleaner:

// This is better
key, elems, err := client.LMPop(ctx, "left", 3, "list1", "list2").Result()

// This is worse
keyVals, err := client.LMPop(ctx, "left", 3, "list1", "list2").Result()

Could we keep the old API or I am missing something?

I like the new KeyValuesCmd name, but I think the old API was cleaner:

// This is better
key, elems, err := client.LMPop(ctx, "left", 3, "list1", "list2").Result()

// This is worse
keyVals, err := client.LMPop(ctx, "left", 3, "list1", "list2").Result()

Could we keep the old API or I am missing something?

OK, we keep the KeyValuesCmd name, but the API uses:

key, elems, err := client.LMPop()

OK, we keep the KeyValuesCmd name, but the API uses:

Sounds good. I think we already have APIs like this.